Skip to content

Comments

Fix multiline environment variable parsing in .env files#928

Merged
rchiodo merged 6 commits intomainfrom
copilot/fix-env-variable-multi-line
Feb 17, 2026
Merged

Fix multiline environment variable parsing in .env files#928
rchiodo merged 6 commits intomainfrom
copilot/fix-env-variable-multi-line

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026 •
edited
Loading

Fix multiline environment variable parsing in .env files

The envFile parser was truncating quoted multiline values at the first newline. A value like:

EXAMPLE_VAR='very long value
with new line , we need to get all the lines'

Would parse as 'very long value instead of preserving the full multiline content.

Changes Made:

  • Analyze the current parseEnvFile and parseEnvLine implementation
  • Implement a stateful parser that handles multiline quoted values
    • Uses character-by-character parsing with state tracking for quotes
    • Preserves newlines within quoted values
    • Handles both single and double quotes
    • Improved escape sequence handling to properly count consecutive backslashes
  • Add comprehensive tests for multiline environment variables
    • Single-quoted multiline values
    • Double-quoted multiline values
    • Escaped newlines (\n)
    • Multiple multiline variables
    • Mix of multiline and single-line values
    • Trailing whitespace preservation
  • Verify the fix works with the example from the issue
    • Tested with actual .env file from the issue
    • Output matches python-dotenv library behavior
  • Address code review feedback
    • Improved escape sequence detection (counts consecutive backslashes)
    • Verified trailing whitespace is preserved correctly
    • Fixed comment about regex 's' flag usage
  • Verified existing tests still pass (lint and build successful)
  • Security check passed (CodeQL found no vulnerabilities)
  • Fixed linter and prettier formatting issues

Solution:

The fix replaces the simple .split('\n') approach with a state machine that:

  1. Tracks whether we're inside quotes (single or double)
  2. Preserves newlines when inside quotes
  3. Only splits on newlines when not inside quotes
  4. Properly handles escaped quotes by counting consecutive backslashes
  5. Resets state properly between lines

This matches the behavior of the python-dotenv library that the issue mentions works correctly.

Testing:

All tests pass, including:

  • Original issue example works correctly
  • Multiline values in single and double quotes
  • Escaped newlines (\n sequences)
  • Multiple multiline variables
  • Trailing whitespace preservation
  • Build and lint successful
  • CodeQL security scan passed with no alerts
  • Prettier formatting applied

Security Summary:

No security vulnerabilities introduced. CodeQL analysis found 0 alerts.

Original prompt

This section details on the original issue you should resolve

<issue_title>Unable to get full environment variable spread on multi line</issue_title>
<issue_description>## Description
Using envFile in launch.json file to load env variables from .env file does not load full variable which have multi line.

Environment Information

  • Python Debugger extension versions 2025.14.1

Example code

import os 

variable = os.getenv("EXAMPLE_VAR")
print("EXAMPLE_VAR:", variable)

launch.json

{
    "version": "0.2.0",
    "configurations": [

        {
            "name": "Python: Debug single file",
            "request": "launch",
            "type": "debugpy",
            "program": "${file}",
            "console": "integratedTerminal",
            "envFile": "${workspaceFolder}/example.env"
        }
    ]
}

example.env

EXAMPLE_VAR='very long value
with new line , we need to get all the lines'

Output

EXAMPLE_VAR: 'very long value

Expected output

EXAMPLE_VAR: very long value
with new line , we need to get all the lines

Using load_dotenv to load .env work as expected

import os 
from dotenv import load_dotenv

load_dotenv(dotenv_path="example.env")
variable = os.getenv("EXAMPLE_VAR")
print("EXAMPLE_VAR:", variable)

## Output
EXAMPLE_VAR: very long value
with new line , we need to get all the lines

Result with env load from launch.json

Image

Result with env load from load_dotenv

Image

<agent_instructions>Investigate if multiline environment variables are passable through DAP and whether or not the extension is expanding them correctly.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@eleanorjboyd is this with or without the python environments extension installed? Can you try disabling it and see if that makes a difference

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits January 15, 2026 00:16
@rchiodo
Fix multiline environment variable parsing in .env files
45a8ecc 
Co-authored-by: rchiodo <19672699+rchiodo@users.noreply.github.com>
@rchiodo
Improve escape sequence handling in environment parser
b673e61 
Co-authored-by: rchiodo <19672699+rchiodo@users.noreply.github.com>
@rchiodo
Update comment for regex 's' flag usage
4bafda0 
Co-authored-by: rchiodo <19672699+rchiodo@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix loading of multi-line environment variables Fix multiline environment variable parsing in .env files Jan 15, 2026
Copilot AI requested a review from rchiodo January 15, 2026 00:23
rchiodo
rchiodo reviewed Jan 15, 2026
View reviewed changes
Copilot AI requested a review from rchiodo January 15, 2026 00:33
@rchiodo rchiodo added the bug Issue identified by VS Code Team member as probable bug label Jan 15, 2026
@rchiodo
Copy link
Contributor

rchiodo commented Jan 15, 2026

@copilot the linter is failing. Please fix the issues found. Prettier also needs to be run.

@rchiodo
Fix linter and prettier formatting issues
e4a6a92 
Co-authored-by: rchiodo <19672699+rchiodo@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 15, 2026

@copilot the linter is failing. Please fix the issues found. Prettier also needs to be run.

Fixed in commit e4a6a92. Applied Prettier formatting to both files - all formatting issues resolved.

@rchiodo rchiodo marked this pull request as ready for review January 15, 2026 00:53
@rchiodo rchiodo requested a review from eleanorjboyd January 15, 2026 00:54
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 15, 2026
edvilme
edvilme reviewed Feb 17, 2026
View reviewed changes
src/extension/common/variables/environment.ts
Copy link

@edvilme edvilme Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this handle cases where it opens with a single quote and closes with a double quote?

Copy link
Contributor

@rchiodo rchiodo Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No that would be invalid. It's why it keeps the quoteChar in memory. It has to match.

@rchiodo rchiodo merged commit 7253c44 into main Feb 17, 2026
18 checks passed
@rchiodo rchiodo deleted the copilot/fix-env-variable-multi-line branch February 17, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rchiodo rchiodo rchiodo approved these changes

@heejaechang heejaechang heejaechang approved these changes

@eleanorjboyd eleanorjboyd Awaiting requested review from eleanorjboyd

+1 more reviewer

@edvilme edvilme edvilme left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@rchiodo rchiodo

Copilot code review Copilot

Labels

bug Issue identified by VS Code Team member as probable bug

Projects

None yet

Milestone

January 2026

Development

Successfully merging this pull request may close these issues.

Unable to get full environment variable spread on multi line

4 participants

@rchiodo @heejaechang @edvilme